Skip to content

[fix] save 추가하여 UserService dirty checking 문제 해결#269

Merged
mannaKim merged 1 commit intodevfrom
feat/user
May 2, 2025
Merged

[fix] save 추가하여 UserService dirty checking 문제 해결#269
mannaKim merged 1 commit intodevfrom
feat/user

Conversation

@mannaKim
Copy link
Copy Markdown
Contributor

@mannaKim mannaKim commented May 2, 2025

🔗 Issue Number

close #256

📝 작업 내역

  • deleteUser, updatePassword, updateUserProfile 메서드에 userRepository.save 추가
  • dirty checking이 동작하지 않아 변경 사항이 DB에 반영되지 않던 문제 해결

💡 PR 특이사항

📸 스크린샷

  • 수정 전 log
    • 비밀번호 변경을 시도했을 때 update가 진행되지 않음
      Image
  • 수정 후 log
    • 비밀번호 변경, 프로필 변경, 탈퇴를 순서대로 진행했을 때의 log
    • 각 실행에서 update가 정상적으로 수행됨
      Image

- deleteUser, updatePassword, updateUserProfile 메서드에 userRepository.save 추가
- dirty checking이 동작하지 않아 변경 사항이 DB에 반영되지 않던 문제 해결
Copy link
Copy Markdown
Contributor

@3uomlkh 3uomlkh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

save() 적용 확인했습니다!

@pshshsh
Copy link
Copy Markdown
Contributor

pshshsh commented May 2, 2025

확인했습니다!

@mannaKim mannaKim merged commit be29d07 into dev May 2, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] User 도메인 유저 정보 변경 시 dirty-checking 실패 대응

3 participants